-
Notifications
You must be signed in to change notification settings - Fork 692
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix crash in DXIL.dll caused by illegal DXIL intrinsic. #6302
Conversation
✅ With the latest revision this PR passed the Python code formatter. |
✅ With the latest revision this PR passed the C/C++ code formatter. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One thing I missed on my last pass, and a cleanup nit.
Replace assert on illegal DXIL op with return illegal value. Check the illegal cases in validation. Fixes microsoft#6168
47a10a8
to
1cd8042
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly speaking up for asserts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good given Chris's responses to my comments.
…osoft#6302) (microsoft#6342)" This file deleted with conflicts from subsequent changes: tools/clang/test/LitDXILValidation/illegalDXILOp.ll This reverts commit 487080f.
Replace assert on illegal DXIL op with return illegal value. Check the illegal cases in validation.
Fixes #6168